-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[3.8] Solve sync build issue #41532
[3.8] Solve sync build issue #41532
Conversation
c1a53dc
to
dcb5144
Compare
dcb5144
to
f1a0698
Compare
Status for workflow
|
🙈 The PR is closed and the preview is expired. |
Status for workflow
|
Status | Name | Step | Failures | Logs | Raw logs | Build scan |
---|---|---|---|---|---|---|
✖ | JVM Tests - JDK 17 | Build |
Failures | Logs | Raw logs | 🚧 |
✔️ | JVM Tests - JDK 21 | Logs | Raw logs | 🚧 |
Full information is available in the Build summary check run.
Failures
⚙️ JVM Tests - JDK 17 #
- Failing: integration-tests/grpc-descriptor-sets/grpc-descriptor-set-alternate-output-dir
📦 integration-tests/grpc-descriptor-sets/grpc-descriptor-set-alternate-output-dir
✖ io.quarkus.grpc.examples.hello.DescriptorSetExistsTest.descriptorSetExists
line 16
- History - More details - Source on GitHub
java.lang.AssertionError:
Expecting path:
/home/runner/work/quarkus/quarkus/integration-tests/grpc-descriptor-sets/grpc-descriptor-set-alternate-output-dir/target/proto
to exist (symbolic links were followed).
at io.quarkus.grpc.examples.hello.DescriptorSetExistsTest.descriptorSetExists(DescriptorSetExistsTest.java:16)
at java.base/java.lang.reflect.Method.invoke(Method.java:568)
at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
@rolfedh I merged it to unblock you but please follow the usual process of pushing to main and we will handle the backport. |
Restructure the conditionalization to avoid breaking the regexp that Michal wrote to transform the tabs, per @gsmet's suggestion.